Skip to content

Massenbearbeitung: Feld-Index um CSRF-Feld versetzt - #1605

Open
tyrant88 wants to merge 1 commit into
yakamara:masterfrom
tyrant88:tyrant88-patch-272
Open

tyrant88 wants to merge 1 commit into
yakamara:masterfrom
tyrant88:tyrant88-patch-272

Conversation

@tyrant88

Copy link
Copy Markdown
Contributor

siehe auch #1601

rex_yform::setCSRFField() setzt das CSRF-Feld per array_unshift an Index 0 der form_elements. getForm() zählt die Felder aber ab 0 und liest bzw. setzt dadurch jeden Wert eine Position zu früh.

Haben alle Datensätze im Feld denselben Wert, schreibt getForm() diesen Wert auf das Häkchen „Feld … bearbeiten“.
Das Häkchen ist dann aus, das Feld wird übersprungen und trotzdem „Datensätze wurde aktualisiert“ gemeldet.
Beispiel: alle Datensätze Status 0, Status auf 1 setzen – nichts passiert.

rex_yform::setCSRFField() setzt das CSRF-Feld per array_unshift an
Index 0 der form_elements. getForm() zählt die Felder aber ab 0 und
liest bzw. setzt dadurch jeden Wert eine Position zu früh.

Haben alle Datensätze im Feld denselben Wert, schreibt getForm() diesen
Wert auf das Häkchen „Feld … bearbeiten“. Das Häkchen ist dann aus, das
Feld wird übersprungen und trotzdem „Datensätze wurde aktualisiert“
gemeldet. Beispiel: alle Datensätze Status 0, Status auf 1 setzen –
nichts passiert.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant